Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

New version: IceFloeTracker v0.6.0 #118004

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

JuliaRegistrator
Copy link
Contributor

@JuliaRegistrator JuliaRegistrator commented Oct 24, 2024

Copy link
Contributor

github-actions bot commented Oct 24, 2024

Hello, I am an automated registration bot. I help manage the registration process by checking your registration against a set of AutoMerge guidelines. If all these guidelines are met, this pull request will be merged automatically, completing your registration. It is strongly recommended to follow the guidelines, since otherwise the pull request needs to be manually reviewed and merged by a human.

1. AutoMerge Guidelines which are not met ❌

  • I was not able to load the package (i.e. import IceFloeTracker failed). See the AutoMerge logs for details.

2. Needs action: here's what to do next

  1. Please try to update your package to conform to these guidelines. The General registry's README has an FAQ that can help figure out how to do so.
  2. After you have fixed the AutoMerge issues, simply retrigger Registrator, the same way you did in the initial registration. This will automatically update this pull request. You do not need to change the version number in your Project.toml file (unless the AutoMerge issue is that you skipped a version number).

If you need help fixing the AutoMerge issues, or want your pull request to be manually merged instead, please post a comment explaining what you need help with or why you would like this pull request to be manually merged. Then, send a message to the #pkg-registration channel in the public Julia Slack for better visibility.

3. To pause or stop registration

If you want to prevent this pull request from being auto-merged, simply leave a comment. If you want to post a comment without blocking auto-merging, you must include the text [noblock] in your comment.

Tip: You can edit blocking comments to add [noblock] in order to unblock auto-merging.

@JuliaRegistrator JuliaRegistrator force-pushed the registrator-icefloetracker-04643c7a-v0.6.0-bf006c3da6 branch from ee72c9e to 4db85d3 Compare October 25, 2024 12:34
JuliaRegistrator referenced this pull request in WilhelmusLab/IceFloeTracker.jl Oct 25, 2024
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-icefloetracker-04643c7a-v0.6.0-bf006c3da6 branch from 4db85d3 to 495f118 Compare October 25, 2024 13:54
JuliaRegistrator referenced this pull request in WilhelmusLab/IceFloeTracker.jl Oct 25, 2024
JuliaRegistrator referenced this pull request in WilhelmusLab/IceFloeTracker.jl Oct 25, 2024
Stick to python 3.11 as Python 3.12 doesn't work
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-icefloetracker-04643c7a-v0.6.0-bf006c3da6 branch from 495f118 to 90faf1f Compare October 25, 2024 15:16
@cpaniaguam
Copy link

cpaniaguam commented Oct 25, 2024

All but the AutoMerge action pass due to use of Python 3.12 by the CI which is not compatible with the versions of our Python deps. Is there a way to change that to Python 3.11? If not, please merge manually. Thanks!

@lgoettgens
Copy link
Contributor

The following dependencies do not have a [compat] entry that is upper-bounded and only includes a finite number of breaking releases: ImageBinarization, TiledIteration

This has to be fixed by you

[noblock]

@cpaniaguam
Copy link

The following dependencies do not have a [compat] entry that is upper-bounded and only includes a finite number of breaking releases: ImageBinarization, TiledIteration

This has to be fixed by you

[noblock]

@lgoettgens That has been fixed. They werent listed in the compat section before but now they are. Please see
https://github.com/WilhelmusLab/IceFloeTracker.jl/blob/ede819aa48550e70484cf45c56dd6fa2cbb5b66d/Project.toml#L71
https://github.com/WilhelmusLab/IceFloeTracker.jl/blob/ede819aa48550e70484cf45c56dd6fa2cbb5b66d/Project.toml#L97

Do they need something like "<2"? The other ones don't seem to need that.

@lgoettgens
Copy link
Contributor

lgoettgens commented Oct 25, 2024

"0" isn't a valid compat string. See the pkg.jl documentation for details. But basically, you have to list all 0.x versions that you are compatible with (since all of these are considered pair wise breaking)

[noblock]

@JuliaRegistrator JuliaRegistrator force-pushed the registrator-icefloetracker-04643c7a-v0.6.0-bf006c3da6 branch from 90faf1f to 5ecc11c Compare October 25, 2024 20:53
JuliaRegistrator referenced this pull request in WilhelmusLab/IceFloeTracker.jl Oct 25, 2024
@GunnarFarneback
Copy link
Contributor

GunnarFarneback commented Oct 25, 2024

[noblock]

Tip: https://github.com/GunnarFarneback/PackageCompatUI.jl can help you set your compat.

@cpaniaguam
Copy link

cpaniaguam commented Oct 25, 2024

Tip: https://github.com/GunnarFarneback/PackageCompatUI.jl can help you set your compat.

@GunnarFarneback @lgoettgens
I think this is good now. Could you please take another look? Thanks for the guidance!

UUID: 04643c7a-9ac6-48c5-822f-2704f9e70bd3
Repo: https://github.com/WilhelmusLab/IceFloeTracker.jl.git
Tree: f3485b68094eb7a683c080c5a48ea35abdf7a1f4

Registrator tree SHA: 17aec322677d9b81cdd6b9b9236b09a3f1374c6a
@JuliaRegistrator JuliaRegistrator force-pushed the registrator-icefloetracker-04643c7a-v0.6.0-bf006c3da6 branch from 5ecc11c to 5ede68c Compare October 25, 2024 21:28
JuliaRegistrator referenced this pull request in WilhelmusLab/IceFloeTracker.jl Oct 25, 2024
@JuliaTagBot JuliaTagBot added the AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. label Oct 26, 2024
@lgoettgens
Copy link
Contributor

[noblock]

a lot of comments above are missing the [noblock] tag

And of course there is still the problem with loading the package

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AutoMerge: last run blocked by comment PR blocked by one or more comments lacking the string [noblock]. BREAKING minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants